Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(temp, iOS): Fix missing weak linking for Xcode 15 #1896

Closed
wants to merge 3 commits into from

Conversation

tboba
Copy link
Member

@tboba tboba commented Sep 20, 2023

Description

Currently it's not possible to build our example apps because of the recent changes in the newest Xcode 15.
In the changelog they're mentioning a new linter, which lacks the support of weak linking.

This PR introduces changes in Podfiles that are required to launch our example apps. They are mainly adding required flag (-ld_classic) that is restoring the support for the classic linter.

This change is temporary! We should observe and wait for the 0.72.5 patch in React Native that adds the support for classic linter inside their Podfiles.

You can check for more information about the newest changes in linter here.

Helpful links

Checklist

  • Ensured that CI passes

@tboba tboba requested review from kkafar and WoLewicki September 20, 2023 16:36
@tboba
Copy link
Member Author

tboba commented Sep 20, 2023

@kkafar @WoLewicki Can you ensure that our example app is being correctly installed and launched on iOS 14 / iOS 16 if you're still using Xcode 14? 🙏

Copy link
Member

@kkafar kkafar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm using latest XCode already.

We can have it merged in case you're working on iOS right now. If it is not the case we can wait a day or two for React Native release, then just bump RN versions in package.json.

One nitpick:

  1. We could also add -Wl option as recommended in official workaround by apple. There is no harm in seeing linker warnings. Up to you.

@kkafar
Copy link
Member

kkafar commented Sep 25, 2023

Hey @tboba, 0.72.5 was released. I believe we can close this PR & open new one - with bumped version of RN.

@tboba
Copy link
Member Author

tboba commented Sep 25, 2023

Hey @kkafar, let's leave this PR open for now - I'm testing RN on the newest version and unfortunately I'm still coming across the EXC_BAD_ACCESS error that happens during the application launch 😕
I'll be providing an updates if anything will change in this area.

@kkafar
Copy link
Member

kkafar commented Sep 25, 2023

KK, I'll try it out soon

@tboba
Copy link
Member Author

tboba commented Oct 30, 2023

As I'm looking on building Example apps on iOS 15 it seems to be fine, also the app is being launched correctly so I assume this PR is no longer relevant.

@tboba tboba closed this Oct 30, 2023
@tboba tboba deleted the @tboba/fix-building-ios-on-xcode-15 branch October 30, 2023 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants